fix(renderers): align numeric table cells with headers - #1051

No actionable comments were generated in the recent review. 🎉

Merge Risk: ⚪ Minimal · up to 133de

Numeric cells now align with their right-aligned headers, with regression coverage for formatting variants and rendered output. No actionable merge risk remains.

• Fix all pre-merge checks with AI

_{Comment @coderabbitai help to get the list of available commands.}

Workflow state fingerprint for Keepalive Loop Reporter. Do not edit.

🤖 Keepalive Loop Status

PR #1051 | Agent: Claude | Iteration 1/12

Current State

Last Claude Run

Claude output:

All checks confirm the implementation matches the documented fix and test coverage is thorough. Now producing the verdict. Verdict: PASS ## Scope Check - Files in diff: 2 - Files matching scope: 2 - Out-of-scope files: none (Scope per issue #1044: src/counter_risk/renderers/table_png.py and `tests...

🧠 Task Analysis

| Provider | 🔶 Regex (fallback) |
| Confidence | 30% |

⚠️ Primary provider (GitHub Models) was unavailable; used Regex (fallback) instead.

Workflow state fingerprint for Agents Gate Followups. Do not edit.

Autofix attempts exhausted for this head.
Attempts: 9 / 3

Latest Gate summary:

Gate run: https://github.com/stranske/Counter_Risk/actions/runs/34685113700
Conclusion: cancelled
PR: #1051
Head SHA: 133def906d57f58b4cae35c7ef0d368721132d19
Autofix attempts for this head: 9 / 3
Fix scope: src/, tests/, tools/, scripts/, agents/, templates/, .github/
Failing jobs:
- classify changed paths (cancelled)

Please investigate manually.

Autofix attempts exhausted for this head.
Attempts: 10 / 3

Latest Gate summary:

Gate run: https://github.com/stranske/Counter_Risk/actions/runs/34685114607
Conclusion: cancelled
PR: #1051
Head SHA: 133def906d57f58b4cae35c7ef0d368721132d19
Autofix attempts for this head: 10 / 3
Fix scope: src/, tests/, tools/, scripts/, agents/, templates/, .github/
Failing jobs:
- Python CI / python 3.12 (cancelled)
  - steps: Restore CI artifact cache (cancelled)
- Python CI / python 3.13 (cancelled)
  - steps: Cache mypy state (cancelled)

Please investigate manually.

Closer audited exact head 133def906d57f58b4cae35c7ef0d368721132d19 against source #1044 and the complete two-file diff. The shared renderer applies the requested right-padding calculation; coordinate and decoded-PNG regressions cover both table variants. The opener recorded 45 passing tests, 8 deliberate-break failures and restored 45 PASS. CodeRabbit reviewed this exact head with no actionable findings; Copilot independently recommends approval on the same head. The optional docstring warning adds no acceptance or correctness debt.

The needs-human label is stale automation state: its two comments cite cancelled Gate runs 34685113700 and 34685114607, not an unresolved product failure or owner decision. Their replacement, Gate 34685167265, completed SUCCESS on this exact head at 09:21 UTC, with both Python versions, Ruff, mypy, conformance and gate-summary passing. Removing the stale label; no retry or additional runner is needed for completed work.

Check-presence audit fetched all 196 check runs and compared them with 234 runs from the recent merged #1043 head. All latest named contexts pass or are intentionally skipped. Six absent nested contexts belong to conditional PR metadata handling and the post-merge verifier; their parent jobs reported SKIPPED, and source/PR routing and body are already complete. They are not missing product tests. Python CI / lint-format is intentionally disabled by format_check: false in the unchanged Gate configuration; Ruff passes. GitHub reports no branch protection and no rulesets, so the lane's explicit gates remain authoritative.

The head has not changed since PR creation at 09:09:38 UTC; the conservative recorded push floor 09:17:32 UTC has elapsed. Full review-thread pagination reports zero active threads. A final unchanged-head, mergeability and check requery precedes merge. After merge, apply verify:compare and retain source #1044 open until its durable report is dispositioned.

Provider Comparison Report

Provider Summary

Agreement

• Verdict: PASS (all providers)
• Correctness: scores within 1 point (avg 9.5/10, range 9.0-10.0)
• Completeness: scores within 1 point (avg 9.5/10, range 9.0-10.0)
• Quality: scores within 1 point (avg 8.5/10, range 8.0-9.0)
• Risks: scores within 1 point (avg 9.5/10, range 9.0-10.0)

Disagreement

Unique Insights

• openai: The renderer now computes data-cell X positions using the same right-boundary and padding calculation as headers whenever column.header_align is "right", while retaining existing left alignment for other columns. This directly satisfies the issue requirement for numeric columns. The added tests i...
• anthropic: The unit test's alignment check assumes column index 0 is always left-aligned and all others right-aligned, an implicit assumption tied to the current _DEFAULT_COLUMNS layout rather than a fully general check of column.header_align.; The deliberate-break gate criterion from the issue (temporarily reverting the alignment formula to confirm the test fails) is a manual verification step and cannot be confirmed from the diff alone, though this does not affect the shipped code.

🔍 LangSmith Traces

• openai
• anthropic

Workflow state fingerprint for Agents Verifier. Do not edit.

Runner dispatch state for claude on PR #1051. Do not edit.